enable nano

A new Pull Request was created by @vlimant (vlimant) for CMSSW_13_0_X.

It involves the following packages:

• PhysicsTools/PatAlgos (xpog, reconstruction)

@cmsbuild, @simonepigazzini, @jfernan2, @mandrenguyen, @vlimant can you please review it and eventually sign? Thanks.
@rappoccio, @gouskos, @Ming-Yan, @hatakeyamak, @emilbols, @mbluj, @ahinzmann, @demuller, @seemasharmafnal, @mmarionncern, @Senphy, @jdolen, @azotz, @jdamgov, @nhanvtran, @gkasieczka, @schoef, @andrzejnovak, @AlexDeMoor, @AnnikaStein, @JyothsnaKomaragiri, @gpetruc, @mariadalfonso this is something you requested to watch as well.
@rappoccio, @antoniovilela, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

• Backported from Change from numberOfMatches() to segment number in DisplacedMuonFilterProducer #41496

FYI @24LopezR

please test

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-86a836/34717/summary.html
COMMIT: 9342710
CMSSW: CMSSW_13_0_X_2023-09-12-1100/el8_amd64_gcc11
Additional Tests: NANO
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/42763/34717/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• You potentially removed 7 lines from the logs
• Reco comparison results: 906 differences found in the comparisons
• DQMHistoTests: Total files compared: 49
• DQMHistoTests: Total histograms compared: 3283545
• DQMHistoTests: Total failures: 4
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3283519
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 48 files compared)
• Checked 213 log files, 164 edm output root files, 49 DQM output files
• TriggerResults: no differences found

NANO Comparison Summary

Summary:

• You potentially removed 6 lines from the logs
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 12
• DQMHistoTests: Total histograms compared: 12436
• DQMHistoTests: Total failures: 0
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 12436
• DQMHistoTests: Total skipped: 0
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 11 files compared)
• Checked 25 log files, 11 edm output root files, 12 DQM output files

Nano size comparison Summary:

changes seem to be consistent with what is expected from #41496 ; @24LopezR can you please confirm ?

Yes, changes are consistent with what is expected.
I had a quick look at the failed comparisons. First, I noticed an increase in DSA Muons (type=8), which is expected. See WF 25202.0:

Also, from numberOfMatches() histogram we observe an increase of muons in the region <2, which is also expected since now the filter is not acting in that region anymore for DSAs. Together with this comes a decrease in the region >=2, because there are DSAs that passed the old filter and now they fail. See comparison from WF 136.793:

These are the same effects seen in #41496

+1

backport of #41496

Same comment as elsewhere, @cms-sw/orp-l2 do we backport to intermediate releases?

type muon

+1

This pull request is fully signed and it will be integrated in one of the next CMSSW_13_0_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_13_3_X is complete. This pull request will now be reviewed by the release team before it's merged. @rappoccio, @antoniovilela, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2)

+1

backport of #41496

Same comment as elsewhere, @cms-sw/orp-l2 do we backport to intermediate releases?

Yes, please backport to the intermediate release cycles.